Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Sync umm_malloc style with upstream #8426

Merged
merged 2 commits into from
Jan 3, 2022

Conversation

mhightower83
Copy link
Contributor

This PR is only style changes.

Upstream umm_malloc at git hash id 4dac43c3be7a7470dd669323021ba238081da18e
processed all project files with the style program uncrustify.

This PR updates our ported version of umm_malloc processed with uncrustify.
This should make subsequent merges of upstream into this port easier.

This also makes the style more consistent through this port..

Upstream umm_malloc at git hash id 4dac43c3be7a7470dd669323021ba238081da18e
processed all project files with the style program uncrustify.

This PR updates our ported version of umm_malloc processed with "uncrustify".
This should make subsequent merges of upstream into this port easier.

This also makes the style more consistant through umm_malloc.
Copy link
Collaborator

@d-a-v d-a-v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for keeping umm up to date !

(sorry - your last commit that I merged first was conflicting with this one)

@d-a-v d-a-v added the merge-conflict PR has a merge conflict that needs manual correction label Jan 3, 2022
@mhightower83
Copy link
Contributor Author

No problem. I wasn't sure how to construct two PRs that would merge well.

@d-a-v d-a-v removed the merge-conflict PR has a merge conflict that needs manual correction label Jan 3, 2022
@d-a-v d-a-v merged commit f26201e into esp8266:master Jan 3, 2022
@mhightower83 mhightower83 deleted the pr-umm_malloc-style branch January 4, 2022 00:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants